Add mission_control-jobs and letter_opener - #1156
Merged
Merged
Conversation
Authorize with the app's own admin check instead of the engine's HTTP basic auth. The engine isolates its namespace, so its controllers resolve bare route helpers against the engine's route set -- hence the thin base controller mirroring Admin::ApplicationController with main_app applied. Also swap Devise::Test::ControllerHelpers for IntegrationHelpers in request specs; the former dereferences a nil `@request`. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Development had no delivery_method set, so mail fell through to :smtp and was silently dropped. Route it to letter_opener_web instead and stop swallowing delivery errors. The engine is only mounted under `Rails.env.local?`, so the gem sits in the development/test group and the navbar link carries the same guard. Run bin/jobs from Procfile.dev too -- PasswordsMailer.reset is enqueued with deliver_later and never reached the mailer without a worker. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
To debugging mail and job related features
screenshots
2026-07-26.141934.mp4
2026-07-26.142207.mp4